Skip to content

Fix!: pass dialect to annotate_types and simplify#4026

Merged
georgesittas merged 1 commit intomainfrom
jo/pass_dialect_to_optimizer_funcs
Mar 26, 2025
Merged

Fix!: pass dialect to annotate_types and simplify#4026
georgesittas merged 1 commit intomainfrom
jo/pass_dialect_to_optimizer_funcs

Conversation

@georgesittas
Copy link
Collaborator

@georgesittas georgesittas commented Mar 25, 2025

Annotations are sensitive to the dialect, e.g. BigQuery overrides ANNOTATORS. Same for simplify, e.g. COALESCE simplification depends on the dialect.

@georgesittas georgesittas requested review from a team and tobymao March 25, 2025 15:29
@georgesittas georgesittas force-pushed the jo/pass_dialect_to_optimizer_funcs branch from ae3b50a to a316530 Compare March 25, 2025 15:30
@georgesittas georgesittas force-pushed the jo/pass_dialect_to_optimizer_funcs branch from a316530 to 7bf4a82 Compare March 25, 2025 17:27
@georgesittas georgesittas force-pushed the jo/pass_dialect_to_optimizer_funcs branch from 7bf4a82 to b174865 Compare March 25, 2025 19:34
@georgesittas georgesittas merged commit 633d8c2 into main Mar 26, 2025
22 checks passed
@georgesittas georgesittas deleted the jo/pass_dialect_to_optimizer_funcs branch March 26, 2025 10:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants